Skip to content

[CI only] Validate Dijkstra grid index units (#1318) - #2

Draft
huxiaoyi-ovo wants to merge 3 commits into
ci/dijkstra-1318-upstream-basefrom
fix/dijkstra-grid-index-units-1318
Draft

huxiaoyi-ovo wants to merge 3 commits into
ci/dijkstra-1318-upstream-basefrom
fix/dijkstra-grid-index-units-1318

Conversation

@huxiaoyi-ovo

@huxiaoyi-ovo huxiaoyi-ovo commented Oct 12, 2026 •

Copy link
Copy Markdown
Owner

Fork-only Draft PR to validate upstream PythonRobotics AtsushiSakai#1461 against the upstream-based CI branch, not for merging into the fork.

The original index-unit correction now includes an additional end-to-end regression for the start-node key -1 collision: planning(1, 2, 1, 3) must reconstruct both the goal and the start.

Validation target: Linux, macOS, Windows and CodeQL. This is CI-only; the actual contribution is AtsushiSakai#1461.

AI assistance was used for the added test and validation workflow.

Copy link
Copy Markdown
Owner Author

Validation completed successfully: Linux, Windows and macOS full CI (Python 3.14) and CodeQL all passed. The reviewed changes have been submitted upstream as PythonRobotics #1461. Closing this fork-only draft to keep the PR queue clean; the CI run history remains linked from the upstream PR description.

@huxiaoyi-ovo huxiaoyi-ovo reopened this Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant